Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update DataProviderList.md #7568

Merged
merged 5 commits into from
May 2, 2022
Merged

Update DataProviderList.md #7568

merged 5 commits into from
May 2, 2022

Conversation

mshd
Copy link
Contributor

@mshd mshd commented Apr 19, 2022

I would like to add new dataProvider developed by @ogroppo
This DataProvider implementation is optimised for prisma and does not use graphql dependency.

add new dataProvider
docs/DataProviderList.md Outdated Show resolved Hide resolved
@djhi
Copy link
Collaborator

djhi commented Apr 20, 2022

What's the difference between your package and the other two already in the list?

@ogroppo
Copy link

ogroppo commented Apr 20, 2022

Hi, it does not rely on graphql so the less boilerplate and setup, it's also intended to be nextjs friendly!

@djhi
Copy link
Collaborator

djhi commented Apr 20, 2022

I see. In this case, would you mind finding a more explicit title so that users know the difference ? Maybe Prisma REST? Thanks!

@mshd
Copy link
Contributor Author

mshd commented Apr 27, 2022

@djhi We renamed it to Prisma NextJS because it's optimized for NextJs

@mshd
Copy link
Contributor Author

mshd commented Apr 30, 2022

Any update on the PR @djhi ?

@fzaninotto
Copy link
Member

I don't see anything related to NextJs in https://github.com/codeledge/ra-prisma-nextjs/tree/main/packages/ra-data-simple-prisma. Perhaps you should underline the absence of dependency on GraphQL instead.

@mshd
Copy link
Contributor Author

mshd commented May 2, 2022

I don't see anything related to NextJs in https://github.com/codeledge/ra-prisma-nextjs/tree/main/packages/ra-data-simple-prisma. Perhaps you should underline the absence of dependency on GraphQL instead.

Ok good point @fzaninotto, it's optimzed for NextJs but obviously isn't restricted to it. I renamed it to Prisma Simple, to highlight the fact that it's a simpler implementation compared to the other packages.

@vercel vercel bot temporarily deployed to Preview – react-admin May 2, 2022 10:36 Inactive
@fzaninotto
Copy link
Member

I updated the other elements to make the difference clearer

@fzaninotto fzaninotto added this to the 4.0.3 milestone May 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants